Skip to content

Update RAG embedding model filter tag to 'vector-rag'#1854

Merged
SaimMomin12 merged 3 commits intobgruening:masterfrom
arash77:rag-vector-rag-filter
Apr 30, 2026
Merged

Update RAG embedding model filter tag to 'vector-rag'#1854
SaimMomin12 merged 3 commits intobgruening:masterfrom
arash77:rag-vector-rag-filter

Conversation

@arash77
Copy link
Copy Markdown
Contributor

@arash77 arash77 commented Apr 28, 2026

The huggingface data table now uses vector-rag as the free_tag for RAG embedding models instead of embedding.

Ref: usegalaxy-eu/infrastructure-playbook#2020

@arash77
Copy link
Copy Markdown
Contributor Author

arash77 commented Apr 28, 2026

Should we bump the version here? We haven't deployed the Hugging Face data table yet.

@arash77 arash77 mentioned this pull request Apr 28, 2026
Merged
@anuprulez
Copy link
Copy Markdown
Contributor

anuprulez commented Apr 28, 2026

Should we bump the version here? We haven't deployed the Hugging Face data table yet.

It may not be necessary as we don't change any input/output.

@anuprulez
Copy link
Copy Markdown
Contributor

anuprulez commented Apr 28, 2026

Failing test may get fixed by adding skip-version-check tag

@arash77
Copy link
Copy Markdown
Contributor Author

arash77 commented Apr 28, 2026

I don't have the permission to add the skip-version-check label. Could someone with write access add it?

@anuprulez
Copy link
Copy Markdown
Contributor

anuprulez commented Apr 28, 2026 •
edited
Loading

I don't have the permission to add the skip-version-check label. Could someone with write access add it?

me neither. Maybe @pavanvidem or @SaimMomin12 can help?

@SaimMomin12 SaimMomin12 reopened this Apr 30, 2026
@SaimMomin12
Copy link
Copy Markdown
Collaborator

SaimMomin12 commented Apr 30, 2026

@anuprulez Is this tool ready for a merge?

@anuprulez
Copy link
Copy Markdown
Contributor

anuprulez commented Apr 30, 2026

@anuprulez Is this tool ready for a merge?

yes, thanks a lot!

@anuprulez
Copy link
Copy Markdown
Contributor

anuprulez commented Apr 30, 2026 •
edited
Loading

I think it needs a sync with the upstream. @arash77 can you do it? thanks!

@SaimMomin12 SaimMomin12 merged commit 7cd4a3e into bgruening:master Apr 30, 2026
11 checks passed
@SaimMomin12
Copy link
Copy Markdown
Collaborator

SaimMomin12 commented Apr 30, 2026

Thanks @arash77 and @anuprulez 🎉

@arash77 arash77 deleted the rag-vector-rag-filter branch April 30, 2026 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SaimMomin12 SaimMomin12 SaimMomin12 approved these changes

Assignees

No one assigned

Labels

skip-version-check

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@arash77 @anuprulez @SaimMomin12